Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Husky and lint-staged for better codestyle checks on commit #136

Merged
merged 4 commits into from
Jul 25, 2019
Merged

Add Husky and lint-staged for better codestyle checks on commit #136

merged 4 commits into from
Jul 25, 2019

Conversation

eXistenZNL
Copy link
Collaborator

What

Add Husky and lint-staged

Why

This makes it much more seamless to only commit correctly styled javascript. Before this PR files would be cleaned but not staged so files with broken codestyle would still be committed.

@eXistenZNL eXistenZNL requested a review from rick-nu July 14, 2019 09:24
@rick-nu
Copy link
Member

rick-nu commented Jul 15, 2019

Awesome, we need this! 👍

Copy link
Member

@rick-nu rick-nu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the old git-hooks, having double will be problematic 😇

@eXistenZNL
Copy link
Collaborator Author

Took care of the old git-hooks, they are gonez0r now!

@eXistenZNL eXistenZNL requested a review from rick-nu July 24, 2019 21:12
@eXistenZNL eXistenZNL dismissed rick-nu’s stale review July 24, 2019 21:17

Made requested changes

Makefile Show resolved Hide resolved
@eXistenZNL eXistenZNL requested a review from rick-nu July 25, 2019 06:40
@rick-nu
Copy link
Member

rick-nu commented Jul 25, 2019

Works like a charm! 👍 Nice!

@rick-nu rick-nu merged commit bd08733 into FuturePortal:master Jul 25, 2019
@T-888 T-888 mentioned this pull request Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants